Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add code coverage exclusions #9118

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

filzrev
Copy link
Contributor

@filzrev filzrev commented Aug 22, 2023

What's included in this PR

  • Set additional ExcludeFromCodeCoverage attribute.
  • Set Justification property to describe why set this attribute.

This PR intended to fix #9067 issue.
PR(#9102) is not sufficient to suppress indirect changes reported by Codecov.
So I've added additional exclusions

@yufeih yufeih closed this Aug 23, 2023
@yufeih yufeih reopened this Aug 23, 2023
@yufeih yufeih merged commit 190ca6a into dotnet:main Aug 23, 2023
@yufeih yufeih added the engineering Makes the pull request to appear in the "Engineering" section of the next release note label Aug 23, 2023
@filzrev filzrev deleted the chore-add-codecoverage-exclusion2 branch August 24, 2023 02:46
p-kostov pushed a commit to ErpNetDocs/docfx that referenced this pull request Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engineering Makes the pull request to appear in the "Engineering" section of the next release note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request] Suppress Codecov indirect coverage changes reports
2 participants